Add the ability to import/export users via CSV#1971
Conversation
|
@jnptk thanks for creating this Pull Request and helping to improve Plone! TL;DR: Finish pushing changes, pass all other checks, then paste a comment: To ensure that these changes do not break other parts of Plone, the Plone test suite matrix needs to pass, but it takes 30-60 min. Other CI checks are usually much faster and the Plone Jenkins resources are limited, so when done pushing changes and all other checks pass either start all Jenkins PR jobs yourself, or simply add the comment above in this PR to start all the jobs automatically. Happy hacking! |
|
DO NOT MERGE YET Notes:
|
stevepiercy
left a comment
There was a problem hiding this comment.
Minor docs tweaks. Looking good so far. Thank you!
Co-authored-by: Steve Piercy <web@stevepiercy.com>
stevepiercy
left a comment
There was a problem hiding this comment.
This looks really good. Thank you! Please see my suggestions.
https://plonerestapi--1971.org.readthedocs.build/en/1971/endpoints/users.html#create-users-via-csv
Co-authored-by: Steve Piercy <web@stevepiercy.com>
stevepiercy
left a comment
There was a problem hiding this comment.
Docs changes look great! I'm glad you caught the empty roles response. I missed that.
|
@davisagli we set the |
stevepiercy
left a comment
There was a problem hiding this comment.
Roles now appear in the response.
https://plonerestapi--1971.org.readthedocs.build/en/1971/endpoints/users.html#create-users-via-csv
Thank you!
davisagli
left a comment
There was a problem hiding this comment.
we set the Location header with the location of the newly created user, but in the case of a csv one might create more than one user. Should we set the header in this case or abandon it?
No, I don't think it makes sense to redirect to one particular user if multiple users were created.
Co-authored-by: David Glick <david@glicksoftware.com>
Co-authored-by: jnptk <110389276+jnptk@users.noreply.github.com>
davisagli
left a comment
There was a problem hiding this comment.
I think this is just about ready!
src/plone/restapi/tests/http-examples/users_add_csv_format.resp
Outdated
Show resolved
Hide resolved
|
@jenkins-plone-org please run jobs |
|
@jenkins-plone-org please run jobs |
📚 Documentation preview 📚: https://plonerestapi--1971.org.readthedocs.build/